Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extract version into gradle.properties. #193

Merged
merged 1 commit into from
Jul 29, 2022

Conversation

dblock
Copy link
Member

@dblock dblock commented Jul 29, 2022

Signed-off-by: dblock [email protected]

Description

The current construct of putting a version in config/version.txt is not very Java like. Moved the version setting to gradle.properties so it can be manipulated as needed in the future in #190.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dblock dblock requested review from a team and madhusudhankonda as code owners July 29, 2022 15:30
@dblock dblock force-pushed the fix-extract-version branch from 27fa75b to c74c860 Compare July 29, 2022 15:32
Copy link
Collaborator

@VachaShah VachaShah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@@ -0,0 +1 @@
systemProp.version = 2.1.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume the version upper/lower case does not matter here?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right.

@dblock dblock merged commit 2c30d4e into opensearch-project:main Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants